Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using CoAP with Node-RED for Low-Power Devices #2422

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

sumitshinde-84
Copy link
Collaborator

Description

Related Issue(s)

Checklist

  • I have read the contribution guidelines
  • I have considered the performance impact of these changes
  • Suitable unit/system level tests have been added and they pass
  • Documentation has been updated

@sumitshinde-84 sumitshinde-84 changed the title Coap with node red Using CoAP with Node-RED for Low-Power Devices Aug 2, 2024
Copy link
Contributor

github-actions bot commented Aug 2, 2024

Images automagically compressed by Calibre's image-actions

Compression reduced images by 43.5%, saving 19.11 KB.

Filename Before After Improvement Visual comparison
src/blog/2024/08/images/coap-in-node.png 15.71 KB 9.23 KB -41.2% View diff
src/blog/2024/08/images/coap-request-node.png 28.20 KB 15.56 KB -44.8% View diff

1284 images did not require optimisation.

@sumitshinde-84 sumitshinde-84 linked an issue Aug 5, 2024 that may be closed by this pull request
@sumitshinde-84 sumitshinde-84 marked this pull request as ready for review August 6, 2024 05:17
@sumitshinde-84 sumitshinde-84 requested a review from gdziuba August 14, 2024 13:30
@ZJvandeWeg ZJvandeWeg requested review from Steve-Mcl and removed request for gdziuba October 18, 2024 11:56
Copy link
Contributor

@Steve-Mcl Steve-Mcl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall, the article reads will and is nicely written however I have 2 concerns with releasing this material.

  1. Unrealistic "Why does it matter" argument. A Rasperry Pi can only run on batteries for a matter of hours (obviously depending on battery size, RPi type and workload) - however we are talking 10s of hours not days or weeks - just feels a little unrealistic and I dont think we should be publishing that kind of documentation if I am honest.

  2. The table of data rates reads great but there is no reference to how we came to these numbers or proofs. The numbers "feel" a bit "perfect" and without any reference to WireShark or our methods, I am hesitant to include such detail. I would either hope to see hard facts to accompany this OR a slimmed down "potential benefits" kind of data

@sumitshinde-84
Copy link
Collaborator Author

Yes, I agree. There is one more reason why I am not considering publishing this article: the node is not working properly right now, as some of its features do not function correctly.

@sumitshinde-84
Copy link
Collaborator Author

That's why I haven't followed up for its review. I think we should close this PR.

@Steve-Mcl
Copy link
Contributor

Happy to close it if the node package is not working well.

Out of interest, what issues did you find? Did you note if the node package had an issue tracker and what the authors responsiveness was?

@sumitshinde-84
Copy link
Collaborator Author

Happy to close it if the node package is not working well.

Out of interest, what issues did you find? Did you note if the node package had an issue tracker and what the authors responsiveness was?

The author is active in merging dependency bot PRs but has not been adding new features or fixing existing issues. The CoAP node has several problems: it doesn't support IPv6 (even though there is an option for it), the agent functionality isn't working, and it lacks support for multicast and conformability( even this option is provided. Additionally, there's a field to set the timeout, but it doesn't work properly—when you try to input a value, it gets wiped out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blog: CoAp with Node-RED
2 participants